Skip to content

[#529] Fix story card layout and desktop click#532

Merged
realproject7 merged 1 commit intomainfrom
task/529-story-card-fix
Mar 25, 2026
Merged

[#529] Fix story card layout and desktop click#532
realproject7 merged 1 commit intomainfrom
task/529-story-card-fix

Conversation

@realproject7
Copy link
Copy Markdown
Owner

@realproject7 realproject7 commented Mar 25, 2026

Summary

  • Moved author name back outside moleskine cover (below cover, above TVL/rating) — fixes desktop click issue where z-40 author link blocked the card link
  • Moved plot count + NEW badges to bottom of cover
  • Genre tag stays at top of cover
  • Reverted to wrapping cover in <Link> (no stretched link pattern needed now)
  • Layout top-to-bottom: moleskine (genre top, title center, plot+NEW bottom) → author → TVL/rating
  • Updated tests to match restored <Link> wrapper
  • Bumped version to 0.1.7

Test plan

  • Desktop: entire card is clickable to story detail page
  • Author name below card links to /profile/[address]
  • Genre badge at top, plot count + NEW at bottom of cover
  • Mobile layout unchanged
  • All 16 StoryCard tests pass

Fixes #529

🤖 Generated with Claude Code

@realproject7 @claude
[#529] Fix story card layout and desktop click
a3e078e 
- Move author name back outside moleskine (below cover, above TVL/rating)
  so profile link doesn't conflict with card link
- Move plot count + NEW badges to bottom of moleskine cover
- Genre tag stays at top of cover
- Revert to wrapping cover in <Link> (no stretched link pattern needed)
- Layout: moleskine (genre top, title center, plot+NEW bottom) → author → TVL/rating
- Update tests to match restored <Link> wrapper
- Bump version to 0.1.7

Fixes #529

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 25, 2026 9:56am

Request Review

project7-interns
project7-interns approved these changes Mar 25, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE — clean fix for the z-40 desktop click issue. Reverts to simple wrapper, author moved outside cover (no nested link conflict), badges repositioned per spec. Layout: genre top → title center → plot+NEW bottom → author → TVL/rating. Tests updated correctly.

project7-interns
project7-interns approved these changes Mar 25, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The current head matches issue #529: the moleskine cover is the story link again, the author is back outside the cover as a separate profile link, and the badge layout is restored to genre-at-top with plot/NEW at the bottom.

Findings

  • None.

Decision

Approving because the PR resolves the desktop click regression without introducing new link conflicts, and the resulting story-card structure matches the requested layout.

@realproject7 realproject7 merged commit 795c3be into main Mar 25, 2026
5 checks passed
@realproject7 realproject7 deleted the task/529-story-card-fix branch March 25, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@project7-interns project7-interns project7-interns approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix story card — move author outside moleskine, move badges to bottom, fix desktop click

2 participants

@realproject7 @project7-interns